Skip to content

chore: Enable to build c2pa-python with a local c2pa-rs build#277

Merged
tmathern merged 11 commits into
mainfrom
mathern/remove-obsolete-apis-local-build
Jun 8, 2026
Merged

chore: Enable to build c2pa-python with a local c2pa-rs build#277
tmathern merged 11 commits into
mainfrom
mathern/remove-obsolete-apis-local-build

Conversation

@tmathern
Copy link
Copy Markdown
Collaborator

@tmathern tmathern commented Jun 5, 2026

Changes in this pull request

Enables debug builds based on checked out c2pa-rs/c2pa-c-ffi

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@tmathern tmathern self-assigned this Jun 5, 2026
@tmathern tmathern changed the title chore: Enable to builde c2pa-python with a local c2pa-rs build chore: Enable to build c2pa-python with a local c2pa-rs build Jun 5, 2026
@tmathern tmathern requested a review from ok-nick June 5, 2026 16:10
Comment thread scripts/build_local_artifacts.py
Copy link
Copy Markdown
Contributor

@ok-nick ok-nick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this script differ from the normal build workflow? It seems like it should be doing a lot of the the same work?

Comment thread scripts/build_local_artifacts.py Outdated
@tmathern
Copy link
Copy Markdown
Collaborator Author

tmathern commented Jun 5, 2026

How does this script differ from the normal build workflow? It seems like it should be doing a lot of the the same work?

The assumptions differ. This one assumes you have a local c2pa-rs checkout next to this repo. The normal workflow takes/downloads a prebuilt released library (and does no rebuild). So it also asks you to have a local cargo setup (which you may not have if developing only Python code).

@tmathern tmathern merged commit a33b0e5 into main Jun 8, 2026
23 checks passed
@tmathern tmathern deleted the mathern/remove-obsolete-apis-local-build branch June 8, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants